Skip to content

feat: update CI workflows#5

Merged
spalmurray merged 16 commits intomainfrom
spalmurray/ci-workflows
May 2, 2025
Merged

feat: update CI workflows#5
spalmurray merged 16 commits intomainfrom
spalmurray/ci-workflows

Conversation

@spalmurray
Copy link
Collaborator

@spalmurray spalmurray commented Apr 29, 2025

Updates the CI workflows to work in new repo.

@spalmurray spalmurray marked this pull request as ready for review April 29, 2025 17:19
@codecov
Copy link

codecov bot commented Apr 29, 2025 •
edited
Loading

❌ 10 Tests Failed:

Tests completed Failed Passed Skipped
7010 10 7000 60
View the top 3 failed test(s) by shortest run time 
api.temp.calculator.test_calculator::test_divide
Stack Traces | 0.001s run time 
def
                test_divide():
                > assert Calculator.divide(1, 2) == 0.5
                E assert 1.0 == 0.5
                E + where 1.0 = <function Calculator.divide at 0x104c9eb90>(1, 2)
                E + where <function Calculator.divide at 0x104c9eb90> = Calculator.divide
                .../temp/calculator/test_calculator.py:30: AssertionError
api.temp.calculator.test_calculator::test_divide
Stack Traces | 0.001s run time 
def
                test_divide():
                > assert Calculator.divide(1, 2) == 0.5
                E assert 1.0 == 0.5
                E + where 1.0 = <function Calculator.divide at 0x104c9eb90>(1, 2)
                E + where <function Calculator.divide at 0x104c9eb90> = Calculator.divide
                .../temp/calculator/test_calculator.py:30: AssertionError
api.temp.calculator.test_calculator::test_divide
Stack Traces | 0.001s run time 
def
                test_divide():
                > assert Calculator.divide(1, 2) == 0.5
                E assert 1.0 == 0.5
                E + where 1.0 = <function Calculator.divide at 0x104c9eb90>(1, 2)
                E + where <function Calculator.divide at 0x104c9eb90> = Calculator.divide
                .../temp/calculator/test_calculator.py:30: AssertionError

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@spalmurray spalmurray changed the title feat: new CI workflows feat: update CI workflows Apr 29, 2025
@spalmurray spalmurray force-pushed the spalmurray/ci-workflows branch from ebdda55 to 923418e Compare April 29, 2025 19:28
spalmurray
spalmurray commented Apr 29, 2025
View reviewed changes
.github/workflows/ci-job.yml
Copy link
Collaborator Author

@spalmurray spalmurray Apr 29, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we only have tests for codecov-cli at the moment.

spalmurray
spalmurray commented Apr 29, 2025
View reviewed changes
codecov-cli/tests/helpers/test_versioning_systems.py
Copy link
Collaborator Author

@spalmurray spalmurray Apr 29, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

had to update these as they're relative to the git root.

spalmurray
spalmurray commented Apr 29, 2025
View reviewed changes
prevent-cli/pyproject.toml
[project.scripts]
prevent = "prevent_cli.main:run"
preventcli = "prevent_cli.main:run"
sentry-prevent-cli = "prevent_cli.main:run"
Copy link
Collaborator Author

@spalmurray spalmurray Apr 29, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is what we settled on with Rohan and Lindsay. Can change in future if we want.

@spalmurray spalmurray merged commit c281903 into main May 2, 2025
21 checks passed
@spalmurray spalmurray deleted the spalmurray/ci-workflows branch May 2, 2025 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@joseph-sentry joseph-sentry joseph-sentry approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@spalmurray @joseph-sentry